Skip to content

TC-821 Direct owner revocation - #297

Closed
samgbafa wants to merge 20 commits into
mainfrom
skgbafa/tc-821-direct-owner-revoke
Closed

samgbafa wants to merge 20 commits into
mainfrom
skgbafa/tc-821-direct-owner-revoke

Conversation

@samgbafa

@samgbafa samgbafa commented Oct 15, 2026 •

Copy link
Copy Markdown
Contributor

Allow owner-only root CID revoke grants from persisted target abilities. Preserve the requested CID and complete space scopes for alias-effective revoke and status; deny foreign targets.

Base implementation: #291 (TC-1071) 693f539. Tests: historical CACAO 12/12; Rocket scope 12/12; owner root 1/1; historical out-of-space 1/1; SDK real node 3/3; core/node strict Clippy and fmt pass.

DO NOT MERGE before #291 merges.

Linear: TC-821, TC-721.

@samgbafa

Copy link
Copy Markdown
Contributor Author

APPROVE: see TC-821
Reviewed head: 45f80a8 (base 97a2b51). Not a review of any newer #291 head.

Locked M1 tests, strict lint and formatting passed. TC-1071.
Verified workspace, DuckDB, WASM, M1/W5, signer and local runtime checks. TC-1071; TC-1302.
Verified workspace, DuckDB node binaries, WASM, M1/W5 and local runtime checks. TC-1071; TC-1304.
@samgbafa
samgbafa changed the base branch from skgbafa/tc-1071-signature-cid to main October 17, 2026 00:14
@samgbafa
samgbafa marked this pull request as draft October 17, 2026 00:15
@samgbafa samgbafa closed this Oct 17, 2026
@samgbafa samgbafa reopened this Oct 17, 2026
@samgbafa

Copy link
Copy Markdown
Contributor Author

REQUEST_CHANGES

@samgbafa

Copy link
Copy Markdown
Contributor Author

REQUEST_CHANGES: see TC-1302

@samgbafa

Copy link
Copy Markdown
Contributor Author

This PR's remaining behaviours are carried by #357 (TC-821), stacked on #353 (TC-1436) and rebased on current main:

  • owner-activatable urn:cid:<target> revoke grants;
  • revoke and status by a canonical CID with no stored row;
  • exact-CID grants covering every stored encoding.

Everything else here (TC-1071, space-scoped controls) is already on main. #357 carries the tests from this branch plus new ones.

@samgbafa samgbafa closed this Oct 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant